Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync grok patterns with logstash patterns #50381

Merged

Conversation

spinscale
Copy link
Contributor

In order to ensure that logstash and Elasticsearch are able to understand
the same grok patterns, this commit adds a few patterns and changes a few.

Source: https://github.com/logstash-plugins/logstash-patterns-core/tree/master/patterns

In order to ensure that logsash and Elasticsearch are able to understand
the same patterns, this commit adds a few patterns and changes a few.
@spinscale spinscale added the :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP label Jan 7, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Ingest)

@danhermann danhermann self-requested a review January 7, 2020 16:57
Copy link
Contributor

@danhermann danhermann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@spinscale
Copy link
Contributor Author

@elasticmachine update branch

@spinscale spinscale merged commit c978659 into elastic:master Jan 8, 2020
spinscale added a commit that referenced this pull request Jan 8, 2020
In order to ensure that logstash and Elasticsearch are able to understand
the same patterns, this commit adapts to changes in logstash, adds a few
patterns and changes a few.
SivagurunathanV pushed a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
In order to ensure that logstash and Elasticsearch are able to understand
the same patterns, this commit adapts to changes in logstash, adds a few
patterns and changes a few.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >enhancement v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants